Skip to content

Conversation

@soham2560
Copy link
Contributor

@soham2560 soham2560 commented Jun 27, 2025

Signed-off-by: soham2560

Description
There is a Build Error for webots_ros2_control on Rolling as described in #1041. Seems to be from the recent update to 5.0.0 and above for versions (refer ros-controls/ros2_control@151b22e)

After some thought, have decided to also add support for ver5.3 here, as it introduces updated API.

Related Issues
This pull-request fixes issue #1041

Affected Packages
List of affected packages:

  • webots_ros2_control

omichel
omichel previously approved these changes Jul 4, 2025
Copy link
Member

@omichel omichel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@soham2560
Copy link
Contributor Author

soham2560 commented Jul 10, 2025

@omichel can you have another look at this, have added support for ver5.3 as well, refer added ver5.3 support

the failing test seems to be unrelated, will merge after your review.

On a side note, is there going to be a release of webots_ros2_control soon?

@soham2560 soham2560 requested a review from omichel July 10, 2025 08:41
@soham2560 soham2560 self-assigned this Jul 10, 2025
@omichel
Copy link
Member

omichel commented Jul 10, 2025

I have no plan for a new release of webots_ros2, but if you are willing to do so, you are welcome. The procedure is explained here. @lukicdarkoo might help you to do it if needed.

@soham2560 soham2560 merged commit 7169ebb into cyberbotics:master Jul 10, 2025
6 of 7 checks passed
@soham2560 soham2560 deleted the update/hw_ver branch July 10, 2025 12:30
@lukicdarkoo
Copy link
Member

Sorry, I started Kilted and Rolling support for a release here:
#1040
(fixing the issue above and a few more)

I will get back to it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants